fix(nits,#14705): voie 3 B.0 — élargir la borne nommeur au coordinateur (LIFT_OVERRIDE_LOGINS) - #15625

VERDICT: LGTM (vérifié: fichiers base+head téléchargés et comparés localement — exactement 3 sites voie 3 élargis dans analyse(), conditions d'admission préservées verbatim, voies 1/2 intactes, 6 tests neufs lus intégralement ; non exécutés, python absent du conteneur)

[NanoClaw] structural review — fix(nits,#14705): voie 3 B.0 — élargir la borne nommeur au coordinateur, 2 fichiers (check_unaddressed_nits.py +17, test_followup.py +105), head f7e77652.

Vérifié firsthand

• Élargissement chirurgical, confirmé au diff local base d14b1ac0 ↔ head : 4 hunks au total — 1 extension de docstring (collect_followup_lifts, séparation conditions 1-4 / 5-6 documentée) + exactement 3 sites dans analyse() (l.~3869/3904/3926) où namer in (login, pr_author) devient namer in (login, pr_author) or namer in LIFT_OVERRIDE_LOGINS. Le fichier ne contient que 3 occurrences de namer in — toutes trois en voie 3, toutes trois élargies. Les voies 1/2 (_lift_eligible, _approved_lifts_reserve) sont intactes, et LIFT_OVERRIDE_LOGINS = {"myia-ai-01"} (l.134) est préexistante, réutilisée telle quelle — aucune valeur neuve, cohérent avec le « aucune surface neuve » du corps de PR.
• Conditions d'admission préservées verbatim sur les 3 sites : when < t < cutoff (fenêtre), when < info.created_at (review antérieure à l'issue), _issue_references_pr(info, pr_number) (l'issue référence la PR). L'élargissement ne touche que l'identité du nommeur crédité, pas les gardes — la levée reste bornée aux issues de suivi délibérées, post-review, référençant la PR.
• Tests bien conçus (+105/-0, 6 nouveaux) : 3 positifs (issue de suivi du coordinateur lève une réserve Hermes / un blocage / un CHANGES_REQUESTED), 1 négatif (un tiers ne lève pas), 1 de préservation des conditions (issue ne référençant pas la PR → pas de levée), et 1 test de mutation qui vide LIFT_OVERRIDE_LOGINS et vérifie que les faux positifs rougissent — il rebind le global du module et attend blocked is True, donc il lie la lecture au moment de l'appel et attraperait toute régression silencieuse du mécanisme. Tous les helpers (run, resolver, make_issue, at, fixtures HERMES_NIT/REPORT_COORD) préexistent à la base — aucun fixture welding.
• 0 secret, 0 chemin personnel dans les deux fichiers au head.

Notes (non bloquantes)

1. Granularité PR-level de la condition 6 (préexistante, désormais aussi au coordinateur) : _issue_references_pr est vérifié au niveau PR, pas au niveau nit — une seule issue de suivi du coordinateur lève donc toutes les réserves éligibles de la PR (toutes les reviews Hermes antérieures à l'issue). C'est la sémantie déjà en vigueur pour pr_author, cohérente avec l'intention (une issue de suivi = arbitrage global), mais l'élargissement augmente le nombre d'identités capables de le faire : la valeur de confiance du compte coordinateur myia-ai-01 monte d'un cran. À garder en tête si la liste grandit un jour.
2. Aliasing sémantique de LIFT_OVERRIDE_LOGINS : la constante sert désormais deux rôles — acteur OVERRIDE (voie 1, préexistant) et nommeur crédité en voie 3 (neuf). Ajouter une machine à la liste pour un motif voie 1 élargirait silencieusement le crédit voie 3. Une constante distincte (ex. FOLLOWUP_LIFT_LOGINS) ou un commentaire reliant explicitement les deux rôles éviterait la dérive future. Pas un défaut de ce diff — le partage est délibéré et annoncé dans le corps de PR.

Portée (honnêteté) : diff intégral des 2 fichiers vérifié localement, hunk par hunk (le script hôte de 4 286 lignes est inchangé partout ailleurs). Les 6 tests ont été lus statiquement, pas exécutés — python est absent de mon conteneur ; l'exécution CI de la suite reste la preuve runtime. Je n'ai pas rejoué le script complet sur un corpus réel de PR.

Merge = Emerjesse.